Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed to dynamically ignore incorrect achievement data. #567

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

shintaxx
Copy link

Added a function to ignore extraneous data included in the JSON data. The JSON data is expected to be mechanically generated, so modifying the JSON data itself was considered inappropriate.

Added a function to ignore extraneous data included in the JSON data. The JSON data is expected to be mechanically generated, so modifying the JSON data itself was considered inappropriate.
@jogerj
Copy link

jogerj commented Aug 27, 2024

@shintaxx
Copy link
Author

shintaxx commented Sep 9, 2024

I fully understand that my points on this issue are duplicative. On top of that, the moderators continue to ignore it, so I am going out of my way to propose a new approach. I question why you continue to ignore all these points. I suspect there is malice in that.
Some of the other proposed fixes are to modify the original data, the JSON file, but please understand that my approach is to look at the ID and ignore it, so there is no need to update the JSON every time there is an update.

In any case, if you are not willing to take this issue seriously, I will not use this site again. Goodbye.

@inoffensive
Copy link

I'm seconding what @shintaxx has mentioned. While I understand that we're not in the position to demand anything because we're not paying to use the site, it's a bit disheartening to see these still up. It's close to a year since they were noticed. Why haven't they been fixed?

The site so behind on so many relevant elements... Spincrystals haven't been updated. Furniture sets haven't been updated. TCG cards haven't been updated. I'm better off assuming it's dead...

@jogerj
Copy link

jogerj commented Sep 11, 2024

I just triage stuff around here. It's up to @MadeBaruna to apply and merge fixes. The Discord server community is pretty active (despite its lawlessness) and community run alternatives are routinely discussed/under development (check out dval.in)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants